Summary

Target 2 (the editor export/save pipeline). EditorView.writeImageSync chose PNG vs JPEG-0.9 by file extension and inlined both encoders. That JPEG branch duplicated CaptureManager.saveJPEG, and CaptureHistoryStore.writeThumbnail duplicated both the JPEG encoder and the thumbnail sizing math — the latter already extracted as ThumbnailGeometry in #10. So this slice collapses three copies of "encode an NSImage to data" down to one.

New Utils/ImageExportFormat.swift:

enum ImageExportFormat: Equatable { case png; case jpeg(quality: Double)
    static func forPath(_ path: String) -> ImageExportFormat   // .jpg/.jpeg → jpeg(0.9), else png
}
extension NSImage { func encoded(as: ImageExportFormat) -> Data? }   // reuses pngData; one JPEG path

Call sites rewired

• EditorView.writeImageSync — the extension if/inline encoders become image.encoded(as: .forPath(path)).
• CaptureManager.saveJPEG — image.encoded(as: .jpeg(quality: quality)).
• CaptureHistoryStore.writeThumbnail — now uses ThumbnailGeometry.fittedSize(...) for sizing and thumb.encoded(as: .jpeg(quality: 0.7)) for encoding (previously a full re-implementation of both).

Scope / safety

• Behavior-preserving — identical format choice (extension match is still case-insensitive via lowercased()), identical compression factors (0.9 editor / 0.7 thumbnails), identical PNG path via the existing pngData.
• No I/O moved — FileManager writes stay at the call sites.

Tests

ImageExportFormatTests (8 cases): forPath for png/jpg/jpeg, case-insensitive matching, unknown/missing extension → png; and encoded(as:) produces data with the correct PNG (\x89PNG) and JPEG (FFD8FF) signatures.

🤖 Generated with Claude Code

Generated by Claude Code